-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
@wkcn could you help to take a review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you provide some performance data?
Could you give a hint on what kind of performance data can be provided? |
A simple way is to compare the performance with and without your omp pragma and see how much speedup from your parallelization. |
btw, I think you still need to register CUDA version and error out with the hint like "No implementation" |
Hi, @pengzhao-intel, the performance w/o omp is as follows:
Settings for the performance data: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Thank you! LGTM : )
@ciyongch could you take a look as well? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution.
Description
Add RROIAlign operator and test case.
Different from ROI Align, RROI Align uses rotated rois, which is suitable for text detection.
Reference paper: Ma, Jianqi, et al. "Arbitrary-Oriented Scene Text Detection via Rotation Proposals."
IEEE Transactions on Multimedia, 2018.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments